From: Xavier Johnson <krinkodot22@hotmail.com>
Date: Tue, 29 May 2018 04:52:24 +0000 (-0400)
Subject: switcherPopup: Cancel window cycling with Tab
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~929^2~5^2~8
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=4a7082bb0f3ec8f8cc4a200aa974aa20de379f8f;p=gnome-shell.git

switcherPopup: Cancel window cycling with Tab

If the Escape key is used for a window/app cycler/switcher shortcut
(such as "Switch windows directly"), then there is no way to cancel
the switching/cycling operation with the keyboard.

This change allows cancelling such an operation by pressing the Tab
key, but only if Tab is not already being used by the current
switcher/cycler shortcut.

Closes: https://gitlab.gnome.org/GNOME/gnome-shell/issues/315
---

diff --git a/js/ui/switcherPopup.js b/js/ui/switcherPopup.js
index a53b1e6e9..d645f1df9 100644
--- a/js/ui/switcherPopup.js
+++ b/js/ui/switcherPopup.js
@@ -192,7 +192,9 @@ var SwitcherPopup = new Lang.Class({
         if (this._keyPressHandler(keysym, action) != Clutter.EVENT_PROPAGATE)
             return Clutter.EVENT_STOP;
 
-        if (keysym == Clutter.Escape)
+        // Note: pressing one of the below keys will destroy the popup only if
+        // that key is not used by the active popup's keyboard shortcut
+        if (keysym == Clutter.Escape || keysym == Clutter.Tab)
             this.destroy();
 
         return Clutter.EVENT_STOP;